Skip to content

Add Mastodon bridge#16

Merged
poVoq merged 4 commits into
matterbridge-org:masterfrom
selfhoster1312:mastodon2
Nov 25, 2025
Merged

Add Mastodon bridge#16
poVoq merged 4 commits into
matterbridge-org:masterfrom
selfhoster1312:mastodon2

Conversation

@selfhoster1312

Copy link
Copy Markdown
Collaborator

New attempt at fixing linter errors

The context is passed by function argument as recommended by golang.

Deferred error checking is applied, simply printing a warning in case of error.

And errors are typed.

@selfhoster1312

Copy link
Copy Markdown
Collaborator Author

Still a few errors silently discarded that should be logged. Don't merge yet

@lil5

lil5 commented Nov 21, 2025

Copy link
Copy Markdown

https://hemaks.org/posts/why-your-obsession-with-code-linting-is-excessive/

Consider these questions when evaluating your linting setup:

  • Does this rule prevent actual bugs? If not, consider removing it.
  • Would a new team member understand why this rule exists? If you can’t explain it in one sentence, it might be too complex.
  • Are we spending more time configuring tools than solving problems? If yes, you’ve gone too far.
  • Do our linting rules reflect our team’s actual coding standards? Rules that everyone ignores or works around are worse than no rules. Remember, the best code is code that works, is maintainable, and can be understood by your teammates. If your linting setup supports these goals, great. If it becomes an obstacle to them, it’s time to reassess.

@selfhoster1312

Copy link
Copy Markdown
Collaborator Author

Added some helper tweaks in #20. Once it is merged this PR can be rebased and reviewed properly.

@poVoq poVoq merged commit 5d530f0 into matterbridge-org:master Nov 25, 2025
6 checks passed
@poVoq

poVoq commented Nov 25, 2025

Copy link
Copy Markdown
Collaborator

@lil5 could you please test if this is still working as expected? Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants